Open
Conversation
- Create ci.yml: test/typecheck/build on PR and develop push - Rewrite release.yml: auto GitHub Release + npm publish on main push - Remove duplicate publish.yml - Default branch changed to develop Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Determine bump type from conventional commits (feat→minor, fix→patch, breaking→major) - Auto-bump package.json version before release - Skip release commits to prevent infinite loop - Commit version bump back to main Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove auto version bump logic, use package.json version directly - Developer bumps version on develop before merging to main - Skip release if tag already exists (no duplicate releases) - Skip build/test steps early when no release needed Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ci: auto version bump on develop→main merge
The doctor command checks for agentlog in PATH. On CI (Linux) it's not installed globally, causing the test to fail. Add a fake agentlog binary alongside the fake codex binary in the test helper. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the project’s automation around Codex/AgentLog integration tests and consolidates GitHub Actions workflows to run CI and publish releases from the main branch based on package.json versioning.
Changes:
- Extend Codex CLI test setup to also stub an
agentlogbinary inPATH. - Replace tag-triggered npm publish workflow with a
main-push release workflow that creates a GitHub Release and publishes to npm when a new version is detected. - Add a dedicated CI workflow for typecheck/test/build.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| src/tests/cli-codex.test.ts | Expands fake PATH setup to include both codex and agentlog executables for tests. |
| .github/workflows/release.yml | Moves release/publish to run on pushes to main, deriving tag/version from package.json and publishing to npm with provenance. |
| .github/workflows/publish.yml | Removes the standalone tag-based npm publish workflow. |
| .github/workflows/ci.yml | Adds CI workflow for typecheck/test/build on selected branch events. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Comment on lines
+45
to
+46
| for (const name of ["codex", "agentlog"]) { | ||
| const bin = join(binDir, name); |
|
|
||
| on: | ||
| push: | ||
| branches: [develop] |
Comment on lines
7
to
13
| jobs: | ||
| publish: | ||
| release: | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: read | ||
| contents: write | ||
| id-token: write | ||
| steps: |
Comment on lines
+33
to
52
| - uses: oven-sh/setup-bun@v2 | ||
| if: steps.tag_check.outputs.exists == 'false' | ||
| with: | ||
| bun-version: latest | ||
|
|
||
| - name: Install dependencies | ||
| if: steps.tag_check.outputs.exists == 'false' | ||
| run: bun install | ||
|
|
||
| - name: Run tests | ||
| run: bun test | ||
|
|
||
| - name: Run typecheck | ||
| - name: Typecheck | ||
| if: steps.tag_check.outputs.exists == 'false' | ||
| run: bun run typecheck | ||
|
|
||
| - name: Test | ||
| if: steps.tag_check.outputs.exists == 'false' | ||
| run: bun test | ||
|
|
||
| - name: Build | ||
| if: steps.tag_check.outputs.exists == 'false' | ||
| run: bun run build |
* feat: add dry-run support, validate command, and structured errors
- Add src/errors.ts with typed error codes and actionable fix messages
- Add --dry-run flag to init and uninstall commands
- Add validate command for machine-readable health checks
- Replace raw console.error strings with structured error formatting
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: address PR review feedback
- Distinguish VAULT_NOT_FOUND vs VAULT_NOT_OBSIDIAN in validateVault
- Pass target to cmdInitDryRun so --codex/--all dry-run shows correct actions
- Distinguish missing vs corrupted config in cmdValidate
- Type toJsonError return as AgentLogError & { status: "error" }
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* test: add tests for --dry-run and validate command
- init --dry-run: valid vault, nonexistent path, missing arg, plain mode
- uninstall --dry-run: no side effects verification
- validate: all-ok, no-config, missing-hook scenarios
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: resolve TS2352 type cast error in errors test
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
- Replace manual arg parsing with Commander.js - Add structured errors (errors.ts with AgentLogError types) - Add --dry-run flag to init and uninstall commands - Add validate command (machine-readable pass/fail) - Add codex-debug command - Add schema command for agent-friendly CLI discovery - Add detect --format json with --fields filtering - Update CI workflow and add publish workflow Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
…ers (#24) * docs: add omx discord bridge design and plan * feat: replace ses_ prefix with claude_/codex_ source in session dividers Session dividers now show the origin tool (claude or codex) instead of the generic "ses" prefix, making it clear which AI produced each session block in the Daily Note. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Apply suggestions from code review Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix: unify SourceType in types.ts and fix divider regex pairing - Rename Source to SourceType in types.ts as single source of truth - Remove duplicate Source type from daily-note.ts, import from types.ts - Fix divider regex to properly pair [[...]] and (...) delimiters Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * test: add codex source divider test Verify that entries with source: "codex" emit [[codex_...]] dividers and session grouping works correctly. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code